Skip to content

[Form] List Symfony UX form types #16377

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 19, 2022

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented Jan 7, 2022

Ultimately, I think the Symfony UX docs should be completely moved to this repo. But I'm not sure what exactly the plans/ideas are for this.

Let's start by just advertising Symfony UX in the official docs :)

cc @tgalopin @weaverryan

@wouterj wouterj requested a review from xabbuh as a code owner January 7, 2022 17:30
@carsonbot
Copy link
Collaborator

Hey!

Oh no, it looks like you have made this PR towards a branch that is not maintained anymore. :/
Could you update the PR base branch to target one of these branches instead? 4.4, 5.3, 5.4, 6.0, 6.1.

Cheers!

Carsonbot

@carsonbot carsonbot added this to the 5.2 milestone Jan 7, 2022
@wouterj wouterj changed the base branch from 5.2 to 5.3 January 7, 2022 17:30
@wouterj wouterj force-pushed the symfony-ux-form-types branch from 508d542 to f29a641 Compare January 7, 2022 17:31
@wouterj
Copy link
Member Author

wouterj commented Jan 7, 2022

Oh my, we already stopped support for 5.2 😅 Thanks Carson!

@wouterj wouterj changed the title Advertise Symfony UX form types [Form] Advertise Symfony UX form types Jan 7, 2022
@wouterj wouterj added the Form label Jan 7, 2022
@wouterj wouterj modified the milestones: 5.2, 3.3, 5.3 Jan 7, 2022
@wouterj wouterj changed the title [Form] Advertise Symfony UX form types [Form] List Symfony UX form types Jan 7, 2022
@94noni
Copy link
Contributor

94noni commented Jan 8, 2022

we already stopped support for 5.2

and 5.3 soon to be as well ^^

Copy link
Contributor

@94noni 94noni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 for referencing UX inside the main doc, for now it lives really alone (blog post)

@javiereguiluz
Copy link
Member

Totally in favor of listing these form types.

My only question is about organization, especially future-proof organization. Do we want to mix UX types with regular types ... or do we group all UX types in a dedicated section to add some text and link explaining that they need Symfony UX, etc. ?

@wouterj wouterj force-pushed the symfony-ux-form-types branch from f29a641 to 0f820de Compare January 16, 2022 19:40
@wouterj
Copy link
Member Author

wouterj commented Jan 16, 2022

@javiereguiluz I've moved them to their own little section

@javiereguiluz
Copy link
Member

Thanks Wouter.

@javiereguiluz javiereguiluz merged commit 8a2724a into symfony:5.3 Jan 19, 2022
@wouterj wouterj deleted the symfony-ux-form-types branch January 19, 2022 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants